-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update logstash docker to use ubuntu 20.04 base image #13442
Conversation
57c2520
to
a8c0ebc
Compare
This passes CI on |
a8c0ebc
to
02afc19
Compare
Fixes #13438 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am getting locale warnings when starting that I don't recall seeing in the centos image.
❯ docker run docker.elastic.co/logstash/logstash-full:8.1.0-SNAPSHOT
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
docker/templates/Dockerfile.j2
Outdated
RUN for iter in {1..10}; do \ | ||
{% if image_flavor != 'ubi8' -%} | ||
export DEBIAN_FRONTEND=noninteractive && \ | ||
{% endif -%} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was tempted to move this export to outside of the run loop, but that would add an extra RUN layer, so let's just keep it as is 👍
Fixed Tests pass across distributions and architectures: https://logstash-ci.elastic.co/job/elastic+logstash+master+multijob-docker-acceptance/198/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…0.04 base image Backport elastic#13442 to 8.0 branch. Original message: * Update logstash docker to use ubuntu 20.04 base image * Correctly set locale for ubuntu docker image * tiny typo fix: ubunto -> ubuntu Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
…se image (#13490) Backport #13442 to 8.0 branch. Original message: * Update logstash docker to use ubuntu 20.04 base image * Correctly set locale for ubuntu docker image * tiny typo fix: ubunto -> ubuntu Co-authored-by: João Duarte <jsvd@users.noreply.github.com> Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
…20.04 base image Backport elastic#13442 to 7.17 branch. Original message: * Update logstash docker to use ubuntu 20.04 base image * Correctly set locale for ubuntu docker image * tiny typo fix: ubunto -> ubuntu Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
…ase image (#13529) Backport #13442 to 7.17 branch. Original message: * Update logstash docker to use ubuntu 20.04 base image * Correctly set locale for ubuntu docker image * tiny typo fix: ubunto -> ubuntu Co-authored-by: João Duarte <jsvd@users.noreply.github.com> Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
This commit updates the base image of the logstash
full
andoss
docker images to use ubuntu 20.04 as the base image.The
ubi8
image remains unchanged